Skip to content

Hausaugabe 2#65

Open
phuonglinh-le wants to merge 7 commits intoProfWider:masterfrom
phuonglinh-le:master
Open

Hausaugabe 2#65
phuonglinh-le wants to merge 7 commits intoProfWider:masterfrom
phuonglinh-le:master

Conversation

@phuonglinh-le
Copy link

Version 2

@ProfWider ProfWider requested review from DanielWagner87 and ProfWider and removed request for DanielWagner87 October 29, 2020 13:08
@ProfWider ProfWider self-assigned this Oct 29, 2020
Copy link
Owner

@ProfWider ProfWider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Soweit ganz gut, aber die HA verlangt zwei unterschiedliche Bugfixes. Ein Bug, der von zwei Tests dokumentiert wird ist zwar überhaupt nicht schlecht, aber für's Bestehen brauchen Sie noch einen weiteren Bug, den Sie dann durch einen anderen Bugfix lösen.
Sie können diese zwei Commits jedoch einfach zu dem bestehenden PR hinzufügen.

@phuonglinh-le
Copy link
Author

Okay

@phuonglinh-le
Copy link
Author

@ProfWider: Ich habe einen neuen Bug gefunden und gefixt! Hoffentlich passt das jetzt.

Copy link
Owner

@ProfWider ProfWider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Streng genommen, würde ich zwar sagen, dass das kein Bug ist, den Sie da fixen sondern eine neue Funktionalität, die Sie hinzufügen, aber langsam scheint sich das ja rumgesprochen zu haben, dass man damit durchkommt ;-)
Insofern eindeutig bestanden jetzt 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants